[Yiseull] WEEK 01 solutions#2665
Open
Yiseull wants to merge 7 commits into
Open
Conversation
Contributor
📊 Yiseull 님의 학습 현황이번 주 제출 문제
누적 학습 요약
문제 풀이 현황
🤖 이 댓글은 GitHub App을 통해 자동으로 작성되었습니다. 🔢 API 사용량 (gpt-5-nano)
|
Member
|
@Yiseull 린트 오류 조치 부탁드립니다! 
|
![dalestudy[bot]](https://avatars.githubusercontent.com/in/1067270?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
🏷️ 알고리즘 패턴 분석
- 패턴: Hash Map / Hash Set
- 설명: 집합(set) 사용으로 중복 여부를 확인하는 방식으로, 해시를 이용한 빠른 중복 탐지가 핵심 패턴입니다.
📊 시간/공간 복잡도 분석
| 복잡도 | |
|---|---|
| Time | O(n) |
| Space | O(n) |
피드백: 정수 배열의 중복 여부를 빠르게 확인하기 위해 모든 원소를 집합에 담아 크기 비교를 수행합니다.
개선 제안: 현재 구현이 적절해 보입니다.
💡 풀이에 시간/공간 복잡도를 주석으로 남겨보세요!
Contributor
There was a problem hiding this comment.
🏷️ 알고리즘 패턴 분석
- 패턴: Hash Map / Hash Set, Two Pointers
- 설명: 집합을 이용해 연속 부분 수열의 시작점을 찾고, 각 시작점에서 연속 길이를 확장해 최장 길이를 구하는 방식으로 구현되어 있습니다. 해시 셋으로 중복 제거 및 O(n) 평균 시간 복잡도를 확보합니다.
📊 시간/공간 복잡도 분석
| 복잡도 | |
|---|---|
| Time | O(n) |
| Space | O(n) |
피드백: 각 수에 대해 시작점 여부를 확인하고, 시작점인 경우에만 연속 길이를 확장합니다.
개선 제안: 현재 구현이 적절해 보입니다.
💡 풀이에 시간/공간 복잡도를 주석으로 남겨보세요!
Contributor
There was a problem hiding this comment.
🏷️ 알고리즘 패턴 분석
- 패턴: Hash Map / Hash Set, Heap / Priority Queue
- 설명: 코드에서 각 숫자의 빈도수를 해시 맵에 저장하고, 빈도에 기반해 최대 k개를 뽑기 위해 힙(우선순위 큐)을 사용합니다. 이로써 두 가지 패턴이 모두 적용됩니다.
📊 시간/공간 복잡도 분석
| 복잡도 | |
|---|---|
| Time | O(n log k) |
| Space | O(n) |
피드백: 해시맵으로 빈도수를 구하고 최소 힙을 이용해 k개를 유지하는 방식으로 동작합니다.
개선 제안: 현재 구현이 적절해 보입니다.
💡 풀이에 시간/공간 복잡도를 주석으로 남겨보세요!
Contributor
There was a problem hiding this comment.
🏷️ 알고리즘 패턴 분석
- 패턴: Hash Map / Hash Set, Two Pointers
- 설명: 주어진 코드는 한 번의 순회로 각 숫자의 필요 보수를 해시 맵에 저장하고, 필요한 값이 이미 맵에 있을 때를 찾아 두 인덱스를 반환한다. 즉 해시 맵을 이용한 탐색 기반 패턴과 두 수의 합 문제의 일반적 해결법을 적용한다.
📊 시간/공간 복잡도 분석
| 복잡도 | |
|---|---|
| Time | O(n) |
| Space | O(n) |
피드백: 가치-인덱스 매핑을 이용해 목표 차이를 가진 숫자를 찾습니다.
개선 제안: 현재 구현이 적절해 보입니다.
💡 풀이에 시간/공간 복잡도를 주석으로 남겨보세요!
parkhojeong
approved these changes
Jun 27, 2026
parkhojeong
left a comment
parkhojeong
left a comment
Contributor
There was a problem hiding this comment.
가독성이 높아서 쉽게 잘 읽혔던 거 같습니다. 수고하셨습니다!
| @@ -0,0 +1,3 @@ | |||
| class Solution: | |||
| def containsDuplicate(self, nums: List[int]) -> bool: | |||
| return len(nums) != len(set(nums)) | |||
Comment on lines
+7
to
+14
| # counter = {} | ||
| # for num in nums: | ||
| # if num in counter: | ||
| # counter[num] += 1 | ||
| # continue | ||
| # counter[num] = 1 | ||
| # | ||
| # return [key for key, counter in sorted(counter.items(), key=lambda x: -x[1])[0:k]] |
Contributor
There was a problem hiding this comment.
중복되는 주석이 들어간 거 같네요. 지워주셔도 좋을거 같습니다!
Comment on lines
+6
to
+14
| if num - 1 in numsSet: | ||
| continue | ||
|
|
||
| size = 1 | ||
| while num + 1 in numsSet: | ||
| size += 1 | ||
| num += 1 | ||
|
|
||
| answer = max(answer, size) |
Contributor
There was a problem hiding this comment.
이해가 아주 잘되는 코드인 거 같습니다 👍
Comment on lines
+18
to
+21
| if num in counter: | ||
| counter[num] += 1 | ||
| continue | ||
| counter[num] = 1 |
Contributor
There was a problem hiding this comment.
Suggested change
| if num in counter: | |
| counter[num] += 1 | |
| continue | |
| counter[num] = 1 | |
| counter[num] = counter.get(num, 0) + 1 |
fallback 값을 사용해볼 수도 있을 거 같네요 :)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
답안 제출 문제
작성자 체크 리스트
In Review로 설정해주세요.검토자 체크 리스트
Important
본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!